Skip to content

docs: add devnote for evaluation replace - #239

Merged
memadi-nv merged 15 commits into
mainfrom
memadi/docs/evaluation-anonymizer-replace-devnote
Aug 11, 2026
Merged

docs: add devnote for evaluation replace#239
memadi-nv merged 15 commits into
mainfrom
memadi/docs/evaluation-anonymizer-replace-devnote

Conversation

@memadi-nv

Copy link
Copy Markdown
Contributor

Summary

Adds a developer note explaining Replace-mode evaluation, its strategy-specific scores, output columns, usage examples, and display_record() report. Includes a report screenshot.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Refactoring
  • CI, release, or contributor workflow update

Contributor Checklist

  • PR title follows Conventional Commits, for example fix: handle empty entity list
  • Related issue is linked, or a maintainer-owned no-issue reason is documented above
  • For non-trivial changes, a plan document is linked above, or the no-plan reason is documented above
  • Public API impact checked; skills/anonymizer/SKILL.md updated if needed
  • No real PII added to tests, docs, notebooks, fixtures, or artifacts
  • No API keys, service tokens, private keys, credentials, or real endpoint secrets added

Validation

  • Commands run:
  • Skipped checks or known failures:

Documentation and Artifacts

  • Docs updated, or not needed
  • If docs changed: make docs-build passes locally
  • If tutorial sources changed: notebooks regenerated with make convert-notebooks
  • If e2e, benchmark, or model-provider behavior changed: relevant validation is listed above

Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
@memadi-nv
memadi-nv requested review from a team as code owners August 7, 2026 00:12
@memadi-nv memadi-nv changed the title Memadi/docs/evaluation anonymizer replace devnote docs: add devnote for evaluation replace Aug 7, 2026
@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds a developer note describing Replace-mode evaluation and its strategy-specific reporting.

  • Documents saving and reevaluating complete anonymization results.
  • Explains entity coverage, optional detection validity, and Substitute-only quality scores.
  • Adds report imagery, author metadata, table styling, and site navigation.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
docs/devnotes/posts/evaluation-anonymizer-replace.md Adds the Replace evaluation developer note; the previously questioned entity-coverage and detection-validity APIs match current HEAD.
docs/css/style.css Prevents evaluation output-column identifiers from wrapping in the new documentation tables.
mkdocs.yml Adds the new developer note to the documentation navigation.
docs/devnotes/.authors.yml Registers the developer note author metadata.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
    A[Replace result] --> B[Anonymizer.evaluate]
    B --> C[Entity coverage]
    B --> D[Optional detection validity]
    B --> E[Substitute quality judges]
    C --> F[DataFrame and display_record report]
    D --> F
    E --> F
Loading

Reviews (8): Last reviewed commit: "prevent output column names from wrappin..." | Re-trigger Greptile

Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>

@lipikaramaswamy lipikaramaswamy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I assume the image and page render correctly when you run the docs locally with make docs-serve or make docs-build?

Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
@memadi-nv

memadi-nv commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Looks good! I assume the image and page render correctly when you run the docs locally with make docs-serve or make docs-build?

confirmed. This has been reviewed locally with make docs-serve.
Update: I noticed that some table column names were wrapping across lines. This commit keeps them on a single line.

Signed-off-by: memadi <memadi@nvidia.com>
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md
Comment thread docs/devnotes/posts/evaluation-anonymizer-replace.md Outdated
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
Signed-off-by: memadi <memadi@nvidia.com>
@memadi-nv
memadi-nv merged commit ce4e3bd into main Aug 11, 2026
16 checks passed
@memadi-nv
memadi-nv deleted the memadi/docs/evaluation-anonymizer-replace-devnote branch August 11, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants